Skip to content

[payloads] feat(scripts): updating manifest generation for JSON:API (#7)#8

Closed
guzmud wants to merge 1 commit into
mainfrom
feat/7-support-new-jsonapi-format
Closed

[payloads] feat(scripts): updating manifest generation for JSON:API (#7)#8
guzmud wants to merge 1 commit into
mainfrom
feat/7-support-new-jsonapi-format

Conversation

@guzmud

@guzmud guzmud commented May 12, 2026

Copy link
Copy Markdown
Member

Proposed changes

  • Updating the generate_manifest.py to handle both the legacy flat JSON format and the new JSON:API format
  • Add requirements for the script (in order to use json-api-doc for JSON:API deserialization and orjson for performances and security)

Testing Instructions

  1. Run the script from the branch on new JSON:API formatted exports in a folder

Related issues

Checklist

  • I consider the submitted work as finished
  • I tested the code for its functionality
  • I wrote test cases for the relevant uses case
  • I added/update the relevant documentation (either on github or on notion)
  • Where necessary I refactored code to improve the overall quality
  • For bug fix -> I implemented a test that covers the bug

@guzmud guzmud force-pushed the feat/7-support-new-jsonapi-format branch from db12f61 to 6c72cd9 Compare May 12, 2026 14:42
@guzmud

guzmud commented May 13, 2026

Copy link
Copy Markdown
Member Author

Adding do not merge since the product orientation of the overall feature in unclear in its expectations regarding implementation (same goes for OpenAEV-Platform/collectors#358). PR in standby in order to have groundwork already done.

@guzmud guzmud closed this May 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

do not merge Do not merge until this label is removed.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

refactor: simplify the behavior of the payloads repository following the new payload export format

1 participant